Normalize SIMD pipelines using TensorPrimitives_ - #3161
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR normalizes several SIMD-heavy pipelines around a new internal TensorPrimitives_ compatibility API (mirroring System.Numerics.Tensors.TensorPrimitives) and removes older, parallel SIMD implementations in favor of a single runtime-shaped dispatch model. It also adds targeted tests and benchmarks to validate correctness across scalar/SIMD width boundaries and to track performance impacts.
Changes:
- Introduce and adopt
TensorPrimitives_-style span operations for common add/multiply/divide/negate/max workloads across codecs and processing helpers. - Refactor/normalize affine
Vector4conversion traversal and pixel blender traversal using operator-based SIMD dispatch, and remove legacy JPEG converter SIMD class hierarchy. - Add tests/benchmarks covering SIMD boundary behavior for PNG filters, ICC LUT normalization, pixel blending, convolution kernels, and Vector4 affine transforms.
Reviewed changes
Copilot reviewed 109 out of 111 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/ImageSharp.Tests/Processing/Processors/Convolution/ConvolutionProcessorHelpersTest.cs | Adds regression test for Gaussian sharpen kernel correctness across sizes. |
| tests/ImageSharp.Tests/PixelFormats/Vector4ConvertersTests.cs | New tests validating affine Vector4 conversion traversal across hardware-width boundaries. |
| tests/ImageSharp.Tests/PixelFormats/PixelBlenderTests.cs | Strengthens blender tests to cover multiple bulk overloads, widths, and coverage/amount variants. |
| tests/ImageSharp.Tests/Metadata/Profiles/ICC/Various/IccLutTests.cs | New tests asserting bit-exact ICC LUT normalization for byte/ushort inputs (including exhaustive ushort). |
| tests/ImageSharp.Tests/Formats/Png/PngEncoderFilterTests.cs | Adds encoder/reference comparisons across SIMD boundaries and pixel strides; minor formatting tidy. |
| tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj | Adds conditional net10 dependency on System.Numerics.Tensors for benchmarks. |
| tests/ImageSharp.Benchmarks/General/PixelConversion/Vector4AffineTransform.cs | New benchmark for operator-driven affine Vector4 transforms. |
| tests/ImageSharp.Benchmarks/Codecs/Png/PngFilterEncode.cs | New benchmark for shared PNG filter encode traversal. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/YccKColorConverter.cs | Updates benchmark to use adaptive converter selection rather than per-width implementations. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/YCbCrColorConversion.cs | Updates benchmark to use adaptive converter selection rather than per-width implementations. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/RgbColorConversion.cs | Updates benchmark to use adaptive converter selection rather than per-width implementations. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/JpegColorPackingScalar.cs | New preserved scalar JPEG packing loops for before/after comparison. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/JpegColorPacking.cs | New benchmark comparing scalar packing vs SIMD transpose implementations. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/GrayscaleColorConversion.cs | Updates benchmark to use adaptive converter selection rather than per-width implementations. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/ColorConversionBenchmark.cs | Fixes benchmark setup to actually populate allocated buffers. |
| tests/ImageSharp.Benchmarks/Codecs/Jpeg/ColorConversion/CmykColorConversion.cs | Updates benchmark to use adaptive converter selection rather than per-width implementations. |
| tests/ImageSharp.Benchmarks/Bulk/ShuffleByte4Channel.cs | Updates shuffle benchmark call to match new shuffle API shape. |
| tests/ImageSharp.Benchmarks/Bulk/Shuffle4Slice3Channel.cs | Updates shuffle benchmark to generic shuffle operators, removing control instance. |
| tests/ImageSharp.Benchmarks/Bulk/Shuffle3Channel.cs | Updates shuffle benchmark to generic shuffle operators, removing control instance. |
| tests/ImageSharp.Benchmarks/Bulk/Pad3Shuffle4Channel.cs | Updates pad+shuffle benchmark to generic shuffle operators, removing control instance. |
| src/ImageSharp/Processing/Processors/Normalization/HistogramEqualizationProcessor{TPixel}.cs | Replaces scalar histogram add loop with TensorPrimitives_ add. |
| src/ImageSharp/Processing/Processors/Convolution/Parameters/BokehBlurKernelDataProvider.cs | Normalizes complex-kernel scaling via TensorPrimitives_ over a flattened float span. |
| src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessorHelpers.cs | Replaces per-element kernel normalization/inversion loops with TensorPrimitives_ operations. |
| src/ImageSharp/PixelFormats/Utils/Vector4Converters.AffineOperators.cs | Adds stateful operator definitions for affine Vector4 transforms across register widths. |
| src/ImageSharp/PixelFormats/Utils/Vector4Converters.Affine.cs | Refactors affine traversal into a single generic operator-driven Apply loop and makes APIs public. |
| src/ImageSharp/PixelFormats/PixelBlenders/IPixelBlenderOperator.cs | New static-abstract operator contract for shared blender traversal across SIMD widths. |
| src/ImageSharp/PixelFormats/PixelBlenders/AssociatedAlphaPixelBlender{TPixel,TOperator}.cs | Refactors associated-alpha blender to use the new operator-based blender base and clamps amount. |
| src/ImageSharp/Metadata/Profiles/ICC/Various/IccLut.cs | Routes LUT normalization through a dedicated normalizer helper. |
| src/ImageSharp/Formats/Webp/Lossless/Vp8LHistogram.cs | Replaces AVX2-only add loop with TensorPrimitives_ add over sliced spans and removes old helper. |
| src/ImageSharp/Formats/Webp/AlphaDecoder.cs | Replaces SIMD/scalar branches with TensorPrimitives_ byte add for vertical unfiltering. |
| src/ImageSharp/Formats/Png/Filters/SubFilter.cs | Replaces bespoke encode loop with shared PngFilterEncoder traversal. |
| src/ImageSharp/Formats/Png/Filters/AverageFilter.cs | Replaces bespoke encode loop with shared PngFilterEncoder traversal. |
| src/ImageSharp/Formats/Jpeg/Components/Encoder/ComponentProcessor.cs | Replaces manual SIMD/scalar sum/multiply loops with TensorPrimitives_ add/multiply. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverterVector512.cs | Removes legacy Vector512 converter base class. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverterVector256.cs | Removes legacy Vector256 converter base class. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverterVector128.cs | Removes legacy Vector128 converter base class. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverterVector.cs | Removes legacy System.Numerics.Vector converter base class. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverterScalar.cs | Removes legacy scalar converter base class. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YccKVector512.cs | Removes legacy YccK Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YccKVector256.cs | Removes legacy YccK Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YccKVector128.cs | Removes legacy YccK Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YccKScalar.cs | Removes legacy YccK scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YCbCrVector512.cs | Removes legacy YCbCr Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YCbCrVector256.cs | Removes legacy YCbCr Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YCbCrVector128.cs | Removes legacy YCbCr Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.YCbCrScalar.cs | Removes legacy YCbCr scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffYccKVector256.cs | Removes legacy TIFF YccK Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffYccKVector128.cs | Removes legacy TIFF YccK Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffYccKScalar.cs | Removes legacy TIFF YccK scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffCmykVector512.cs | Removes legacy TIFF CMYK Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffCmykVector256.cs | Removes legacy TIFF CMYK Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffCmykVector128.cs | Removes legacy TIFF CMYK Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.TiffCmykScalar.cs | Removes legacy TIFF CMYK scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.RgbVector512.cs | Removes legacy RGB Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.RgbVector256.cs | Removes legacy RGB Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.RgbVector128.cs | Removes legacy RGB Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.RgbScalar.cs | Removes legacy RGB scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.RgbOperator.cs | Adds operator-based RGB converter implementation (scalar + SIMD lanes + ICC path). |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.GrayScaleVector512.cs | Removes legacy grayscale Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.GrayScaleVector256.cs | Removes legacy grayscale Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.GrayScaleVector128.cs | Removes legacy grayscale Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.GrayScaleScalar.cs | Removes legacy grayscale scalar implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.CmykVector512.cs | Removes legacy CMYK Vector512 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.CmykVector256.cs | Removes legacy CMYK Vector256 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.CmykVector128.cs | Removes legacy CMYK Vector128 implementation. |
| src/ImageSharp/Formats/Jpeg/Components/ColorConverters/JpegColorConverter.CmykScalar.cs | Removes legacy CMYK scalar implementation. |
| src/ImageSharp/Common/Helpers/Vector512Utilities.cs | Adds portable Vector512<byte> subtract-saturate helper mirroring runtime behavior. |
| src/ImageSharp/Common/Helpers/Vector256Utilities.cs | Aligns subtract-saturate fallback behavior/comments with runtime portable approach. |
| src/ImageSharp/Common/Helpers/Vector128Utilities.cs | Simplifies unsigned byte subtract-saturate fallback to a min/sub identity. |
| src/ImageSharp/Common/Helpers/TensorPrimitives_.Multiply.cs | Adds generic operator-based multiply traversal (scalar + 128/256/512). |
| src/ImageSharp/Common/Helpers/TensorPrimitives_.Divide.cs | Adds divide traversal with restricted vectorization rules to match runtime capabilities. |
| src/ImageSharp/Common/Helpers/TensorPrimitives_.Add.cs | Adds generic operator-based add traversal (span+span and span+scalar). |
| src/ImageSharp/Common/Helpers/Shuffle/IShuffle3.cs | Replaces stateful shuffle control with stateless operator (ZYXShuffle3) + intrinsic helper. |
| src/ImageSharp/Common/Helpers/Shuffle/IComponentShuffle.cs | Refactors shuffle contract to static-abstract scalar/vector operators. |
| src/ImageSharp/ColorProfiles/ColorProfileConverterExtensionsIcc.cs | Replaces ad-hoc SIMD/scalar loops with TensorPrimitives_ max/multiply over flattened Vector4 spans. |
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 109 out of 111 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
tests/ImageSharp.Benchmarks/Codecs/Png/PngFilterEncode.cs:87
AverageFilter.Encodeexpectsuint bytesPerPixel, but this benchmark passesBytesPerPixelas anint, which will not compile once theEncodesignature is updated. Cast the constant touintto match the API (consistent with other call sites in the PR).
This was referenced Aug 21, 2026
IhateTrains
added a commit
to ParadoxGameConverters/ImperatorToCK3
that referenced
this pull request
Aug 23, 2026
Updated [SixLabors.ImageSharp](https://github.com/SixLabors/ImageSharp) from 3.1.12 to 4.1.1. <details> <summary>Release notes</summary> _Sourced from [SixLabors.ImageSharp's releases](https://github.com/SixLabors/ImageSharp/releases)._ ## 4.1.1 ## What's Changed * Fix CCITT decompressor bounds checks and error handling by @JimBobSquarePants in SixLabors/ImageSharp#3176 **Full Changelog**: SixLabors/ImageSharp@v4.1.0...v4.1.1 ## 4.1.0 ## What's Changed * GIF: background handling & quantizer overflow fix by @JimBobSquarePants in SixLabors/ImageSharp#3133 * Validate PBM max pixel value by @JimBobSquarePants in SixLabors/ImageSharp#3134 * Add support for Apple CgBI PNG images by @Erik-White in SixLabors/ImageSharp#3136 * Fix GIF transparency handling and dither by @JimBobSquarePants in SixLabors/ImageSharp#3143 * Bump codecov/codecov-action from 6 to 7 by @dependabot[bot] in SixLabors/ImageSharp#3141 * Add BlendWithCoverage overloads. Optimize Rgba32 compatible shuffling. by @JimBobSquarePants in SixLabors/ImageSharp#3150 * Bump actions/checkout from 6 to 7 by @dependabot[bot] in SixLabors/ImageSharp#3146 * Bump actions/cache from 5 to 6 by @dependabot[bot] in SixLabors/ImageSharp#3149 * Fix convolution sampling for bounds smaller than the kernel radius by @JimBobSquarePants in SixLabors/ImageSharp#3152 * Enhance pixel formats and associated-alpha operations with optimizations by @JimBobSquarePants in SixLabors/ImageSharp#3154 * Bump actions/setup-dotnet from 5 to 6 by @dependabot[bot] in SixLabors/ImageSharp#3155 * Fix degeneracy check for resize transforms by @JimBobSquarePants in SixLabors/ImageSharp#3157 * Improvements to Apple CgBI PNG handling by @Erik-White in SixLabors/ImageSharp#3137 * Clone TIFF profiles into image metadata by @JimBobSquarePants in SixLabors/ImageSharp#3159 * Add AOT compiler seeds for pixel formats and ICO, CUR, QOI codecs by @JimBobSquarePants in SixLabors/ImageSharp#3160 * Normalize SIMD pipelines using TensorPrimitives_ by @JimBobSquarePants in SixLabors/ImageSharp#3161 * Add ANI decoder support by @Poker-sang in SixLabors/ImageSharp#2899 * Remove AntialiasThreshold from GraphicsOptions by @JimBobSquarePants in SixLabors/ImageSharp#3162 * Add plus and extended pixel blend modes by @JimBobSquarePants in SixLabors/ImageSharp#3163 * Fix 16-bit binary PBM sample byte order by @JimBobSquarePants in SixLabors/ImageSharp#3164 * Make MemoryAllocator limits configurable and extensible by @JimBobSquarePants in SixLabors/ImageSharp#3165 ## New Contributors * @Erik-White made their first contribution in SixLabors/ImageSharp#3136 **Full Changelog**: SixLabors/ImageSharp@v4.0.0...v4.1.0 ## 4.0.0 ## What's Changed * Update to net8 by @stefannikolei in SixLabors/ImageSharp#2583 * Handle dedup of local palette of 256 length - Main by @JimBobSquarePants in SixLabors/ImageSharp#2607 * Replace custom Crc32 by @JimBobSquarePants in SixLabors/ImageSharp#2611 * Sync 3.1 DrawImage fixes by @tocsoft in SixLabors/ImageSharp#2612 * Fix handling gif encoding for global palettes - Main by @JimBobSquarePants in SixLabors/ImageSharp#2615 * Bump actions/setup-dotnet from 3 to 4 by @dependabot[bot] in SixLabors/ImageSharp#2613 * Adjusted the casing of the Webp format name by @jscarle in SixLabors/ImageSharp#2623 * Fix Paeth Filter decode on platforms that do not support Ssse3 - Main by @JimBobSquarePants in SixLabors/ImageSharp#2620 * Fix WebP animation speed bug by @marklagendijk in SixLabors/ImageSharp#2624 * Promote PixelTypeInfo to Pixel by @stefannikolei in SixLabors/ImageSharp#2601 * TGA: Treat 32 bit True Color images always as transparent by @brianpopow in SixLabors/ImageSharp#2643 * Modernize and optimize pixel format operations across platforms. by @JimBobSquarePants in SixLabors/ImageSharp#2645 * Cleanup SimdUtils by @JimBobSquarePants in SixLabors/ImageSharp#2654 * Bump actions/cache from 3 to 4 by @dependabot[bot] in SixLabors/ImageSharp#2648 * Bump codecov/codecov-action from 3 to 4 by @dependabot[bot] in SixLabors/ImageSharp#2657 * Bump NuGet/setup-nuget from 1 to 2 by @dependabot[bot] in SixLabors/ImageSharp#2658 * Add v3.1.x fixes #2673 and #2674 into main. by @JimBobSquarePants in SixLabors/ImageSharp#2675 * Add fixes 2668, 2676, and 2677 to main by @JimBobSquarePants in SixLabors/ImageSharp#2678 * Merge 2681 to v4 Main by @JimBobSquarePants in SixLabors/ImageSharp#2690 * Add JPEG COM marker support by @RobertMut in SixLabors/ImageSharp#2641 * Bump actions/upload-artifact from 3 to 4 by @dependabot[bot] in SixLabors/ImageSharp#2625 * Only exit JPEG scan decoding after multiple EOF hits by @JimBobSquarePants in SixLabors/ImageSharp#2701 * V4 Ensure VP8X alpha flag is updated correctly. by @JimBobSquarePants in SixLabors/ImageSharp#2703 * Fix animated png handling (issue #2708) by @SpaceCheetah in SixLabors/ImageSharp#2710 * Merge latest release from v3 by @JimBobSquarePants in SixLabors/ImageSharp#2720 * Fix MacOS jobs by @antonfirsov in SixLabors/ImageSharp#2728 * Fix async-over-sync issue in Image.DecodeAsync() by @kroymann in SixLabors/ImageSharp#2725 * Fix overflow in MemoryAllocator.Create(options) by @antonfirsov in SixLabors/ImageSharp#2730 * GifDecoder: Limit lzw bits to a maximum of 12 bits by @brianpopow in SixLabors/ImageSharp#2744 * GifDecoder : Allow skipping bad metadata using identify by @JimBobSquarePants in SixLabors/ImageSharp#2749 * Add ICO and CUR file decoder. by @frg2089 in SixLabors/ImageSharp#2579 * v4 - Fix off-by-one error when centering a transform. by @JimBobSquarePants in SixLabors/ImageSharp#2761 * v4 Fix 2758 by @JimBobSquarePants in SixLabors/ImageSharp#2764 * Simplify Color Space Conversion APIs by @JimBobSquarePants in SixLabors/ImageSharp#2739 * Webp: Fix Issue 2763 by @brianpopow in SixLabors/ImageSharp#2767 * V4 Correctly break during Png decoding by @JimBobSquarePants in SixLabors/ImageSharp#2773 * V4 : Fix filtering on PNG encode. by @JimBobSquarePants in SixLabors/ImageSharp#2778 * Fix #2779 buffer overrun by @KirillAldashkin in SixLabors/ImageSharp#2780 * Fix ImageMetadata docs typo by @lofcz in SixLabors/ImageSharp#2781 * Add API for metadata conversion between formats. by @JimBobSquarePants in SixLabors/ImageSharp#2751 * Tiff decoder: Fix issue 2679 by @brianpopow in SixLabors/ImageSharp#2789 * Replace PngCrcChunkHandling by @JimBobSquarePants in SixLabors/ImageSharp#2786 * Add tagname to debugger visualization for Exif- and Iptc-values, to facilitate easier debugging and discovery by @lassevk in SixLabors/ImageSharp#2787 * V4 - Correctly handle transform spaces when building transform matrices. by @JimBobSquarePants in SixLabors/ImageSharp#2795 * Allow decoding Tiff of different frame size. by @JimBobSquarePants in SixLabors/ImageSharp#2788 * Add progressive JPEG encoder by @ardabada in SixLabors/ImageSharp#2740 * Fix using dither in BmpEncoder when bit per pixel is <= 4 by @mistoll in SixLabors/ImageSharp#2819 * Add QuadDistortion to ProjectiveTransformBuilder by @Socolin in SixLabors/ImageSharp#2748 * WEBP : Use Correct Width With AlphaDecoder by @JimBobSquarePants in SixLabors/ImageSharp#2823 ... (truncated) Commits viewable in [compare view](SixLabors/ImageSharp@v3.1.12...v4.1.1). </details> [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: iht <IhateTrains@users.noreply.github.com>
This was referenced Aug 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Prerequisites
Description
This change introduces the internal
TensorPrimitives_compatibility type and uses the runtime's genericoperator-based SIMD model to normalize ImageSharp's span operations. The compatibility API follows
System.Numerics.Tensors.TensorPrimitivesso it can be replaced by the runtime implementation when ImageSharpmoves to .NET 10.
The migration removes the replaced legacy SIMD implementations rather than retaining parallel pipelines. It also
uses portable
Vector128,Vector256, andVector512APIs where the target framework exposes them, withplatform-specific intrinsics isolated behind compatibility helpers only where .NET 8 has no equivalent API.
The normalized implementation follows runtime main's widest-supported dispatch for every supported element type:
AVX-512 is selected when one complete
Vector512<T>fits, and shorter inputs fall through to a width that fits.SIMD layouts, masks, shuffles, overlap handling, scalar equivalence, and width-selection decisions are documented inline.
Performance
The tables below compare the implementation removed by this PR ("Before") with the retained normalized
implementation ("After"). Lower is better.
MemoryDiagnoserwas enabled for every row containing an allocation result.0 B / 0 Bmeans BenchmarkDotNetmeasured zero managed bytes allocated per operation for both the before and after implementations.
Tensor primitive migrations
JPEG packing pipelines
Additional migrations found during the library-wide audit
Byte-add production callers
The PNG rows compare the exact AVX2 loop removed from
UpFilter.Decodewith the productionTensorPrimitives_.Add<byte>call in the same benchmark run. The runtime-shaped implementation has asmall-buffer dispatch cost, crosses over by 2,048 payload bytes, and benefits increasingly from AVX-512 on
larger scanlines.
Binary size
Release
net8.0artifacts were built frommainand this PR using the same .NET SDK 10.0.302 installation andproject settings.
mainSixLabors.ImageSharp.dll)SixLabors.ImageSharp.pdb)The package reduction is smaller than the assembly reduction because the removed generated IL is highly
repetitive and therefore compresses efficiently inside the NuGet package.
NativeAOT binary-size impact
mainSizoscope attributes the remaining increase predominantly to statically specialized SIMD/operator code. Further material sharing would require runtime dispatch, reduced inlining, or eager initialization, potentially harming performance.
Both executables completed the representative processing and codec workload successfully with identical output.